fix: use authMode apiKey for Ollama provider to avoid streamSimple crash (#3440)#3443
Conversation
…ash (gsd-build#3440) authMode 'none' now requires a streamSimple handler in model-registry.ts. Ollama doesn't provide one, causing a crash when opening /model selector. Fix: switch to authMode 'apiKey' with a dummy apiKey value of 'ollama'. Ollama ignores authentication keys entirely, so this has no functional impact but satisfies the new validation in model-registry.ts line 699.
🔴 PR Risk Report — CRITICAL
Affected Systems
File Breakdown
|
|
Modified my local CLI with this change and it resolved an issue I was having with logout. I have a local Qwen3 model that was apparently causing the issue. What's interesting, or annoying, is I was trying to login and logout of my claude code session and this was blocking that. |
|
Thanks for the fix, @Codergoterrors. PR #3425 addresses this same crash but also includes the slash-command extension fallback fix and Ollama UI fixes. Recommending #3425 be merged and this PR closed to avoid duplicate work — the If you'd like to keep contributing, the non-image absolute path interception described in #2926 would be a good follow-on PR. |
|
Closing in favor of #3425, which addresses the same Ollama |
4 participants
Fixes #3440
Problem
model-registry.tsline 699 now requiresstreamSimplewhenauthModeis"none". The Ollama extension registers withauthMode: "none"but nostreamSimplehandler → crash whenopening
/modelselector with Ollama running.Fix
Switch Ollama's provider registration to
authMode: "apiKey"witha dummy
apiKey: "ollama". Ollama ignores authentication entirely,so this has zero functional impact but satisfies the new validation.
Testing